Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[GH-411] Dungeon supercampaign scene #478

Open
wants to merge 15 commits into
base: main
Choose a base branch
from

Conversation

ncontinanza
Copy link
Collaborator

@ncontinanza ncontinanza commented Jun 3, 2024

Part of #411

Motivation

Connect the backend features that implement the dungeon supercampaign, its campaign and levels with our client.

Summary of changes

  • Dungeon supercampaign prefab
  • Dungeon campaign prefab and levels

How has this been tested?

  • Start the game, go to the Dungeon scene and click on the Battle button.
  • Select the dungeon supercampaign
  • Select the first level and start the battle.
  • Check that the battle finishes properly. If you win, the second level should be unlocked.

In order to win the dungeon levels and test all the available levels in the campaign, you may need to edit the character.csv in the backend and reduce El1xir's life to 10, so that you can easily beat it in this first level.

Checklist

  • I have tested the changes locally.
  • I self-reviewed the changes on GitHub, line by line.
  • Tests have been added/updated.
  • This change requires new documentation.
    • Documentation has been added/updated.
  • I have tested the changes in another devices.
    • Tested in iOS.
    • Tested in Android.

@ncontinanza ncontinanza changed the title Gh 411?¡¨* dungeon supercampaign scene [GH-411] Dungeon supercampaign scene Jun 3, 2024
Copy link
Collaborator

@tkz00 tkz00 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When in the dungeon campaigns map screen, the back button takes me to the Overworld, it should take me to Dungeon.

@ncontinanza ncontinanza added the Waiting on PR Use this tag to alert reviewers that this PR shouldn't be merged until another one is merged first label Jun 4, 2024
@lotuuu
Copy link
Collaborator

lotuuu commented Jun 4, 2024

Here's some easier code for fetching the user without needing to know the name!
[user] = GameBackend.Repo.all(GameBackend.Users.User)
assuming you didn't crate any other users of course

@lotuuu lotuuu mentioned this pull request Jun 4, 2024
8 tasks
@tkz00
Copy link
Collaborator

tkz00 commented Jun 7, 2024

Also I think it would be nice that if a level has a max number of units, it automatically deselects the leftover units, but I believe that is a backend change.

@ncontinanza
Copy link
Collaborator Author

ncontinanza commented Jun 10, 2024

Also I think it would be nice that if a level has a max number of units, it automatically deselects the leftover units, but I believe that is a backend change.

Yes, we'll need a PR in the backend for that to be made properly: #485

@ncontinanza ncontinanza removed Waiting on PR Use this tag to alert reviewers that this PR shouldn't be merged until another one is merged first labels Jun 11, 2024
tkz00
tkz00 previously approved these changes Jun 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants